Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

All three brackets for variables #14465

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

All three brackets for variables #14465

wants to merge 1 commit into from

Conversation

JonasHelming
Copy link
Contributor

@JonasHelming JonasHelming commented Nov 16, 2024

fixed #14464

What it does

Allows to use three brackets for variables in prompts. No mixed formats are allows for consistency

How to test

There are automated test.
To manually test,

  • open the code completion prompt and a code file.
  • Add one bracket to {{textUntilCurrentPosition}}, see how it fails.
  • Add the second bracket to complete {{{textUntilCurrentPosition}}} and see how it works again.
  • Check that the variable is still listed in the AI configuration view then.
  • Check that in the prompt editor, three brackets are valid, but mixtures are an error

Review checklist

Reminder for reviewers

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Waiting on reviewers
Development

Successfully merging this pull request may close these issues.

[Theia AI] Allows three brackets for variables in prompt templates
1 participant